Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5.2.3] User Action Logs Email #44709

Open
wants to merge 1 commit into
base: 5.2-dev
Choose a base branch
from

Conversation

brianteeman
Copy link
Contributor

Pull Request for Issue #39328 .

Summary of Changes

  1. the htmlbody string is sanitized by the html editor when it is loaded which is why {messages} is moved outside of the table. This PR moves {messages} to wrap the table instead of the row to avoid this sanitization and make the email work
  2. {messages} is missing from the params field in the database which is why it doesnt appear in the list of tags to insert and thus without that tag nothing is in the email

Testing Instructions Part 1

Set Mail Templates to send mail as html
Enable receiving action log emails for your user and do something to trigger an email eg saving an article
image

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

image

Testing Instructions Part 2

Open the User Actions Log: Notification Mail template

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

image

NOTE: If a user has already saved a User Actions Log: Notification Mail template they will need to press the reset HTML Body button.

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Language Change This is for Translators PR-5.2-dev
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants